Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cody Web: Polish cody web Prompts #6135

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

vovakulikov
Copy link
Contributor

This PR does a few improvements for the prompts library in Cody Web

  • First of all, it fixes recent regression in openctx (apparently internally openctx library tries to find vs code setting file and with an empty Cody Web Workspace URI it fails in runtime and therefore doesn't produce any openctx providers such as remote files or repositories)
  • It fixes prompts editor placeholder appearance (we no longer add whitespaces to the field unconditionally)
  • Disables non-runnable in Cody Web prompts (insert or inline commands) before we render them and these actions did nothing

Test plan

  • Check that you can use openctx mentions in Cody Web (with Cody Web linked in sg, check ./web contribution doc to see how to link Cody Web and sg instance)
  • Check that you can't click non-chat prompts in Cody Web and see that they are disabled
  • Check that you can see the prompts library placeholder if you don't have context (Cody Web standalone page case)

Changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant